; Merge from origin/emacs-25

The following commit was skipped:

9d0b103 Minor fixes in calculator.el

Merge from origin/emacs-25

e6a3819 Update HISTORY section in readme for the NextStep interface.
f67f1ed ; * doc/lispref/modes.texi (Font Lock Basics): Minor rewording.
7c81a0b Improve documentation of 'save-place-mode'
cab3f0a Allocate glyph matrices for the initial frame
e01c72f Fix white space in last checkin
370eb67 Make `insert-pair' always leave the cursor where documented
b594393 etc/NEWS: Mention the new second parameter to `package-install'

; Merge from origin/emacs-25

The following commit was skipped:

ce4bdd7 Highlight assignments in Makefiles more correctly

Merge from origin/emacs-25

bd58c13 Improve documentation of focus-related hooks
00a4720 Further improve doc string of 'disable-point-adjustment'
c582def Further adaptions in file-notify-tests.el for w32notify
a1585e1 Don't bug out on localised dates in gnus-icalendar

; Merge from origin/emacs-25

The following commits were skipped:

dfe1650 (ls-lisp-insert-directory): Make -B work
42bc065 Make buttons in header lines work

Merge from origin/emacs-25

7bc4820 Make setf for frame-height/width work again

; Merge from origin/emacs-25

The following commit was skipped:

1af5e6e Encode header strings before printing

Merge from origin/emacs-25

6620944 (cl-union): Do not ignore :test argument when lists are equal.
17dd3fb Add `isearch' to `basic-faces'
c1ec743 Make $, : and @ "prefix characters" in ruby-mode
e72a26e Make find-tag-default-bounds more strict
1bc0e0a Minor fixes in filenotify.el

Fix issue of inserting images on some systems

* lisp/net/shr.el (shr-tag-img): Construct a non-empty range
to pass to shr-image-fetched, to indicate where to insert the
image.  Fixes the issue introduced with commit
80852f843e69b81618f29cfb9aa4b074946cb3c4
(bug#22789).

Ensure TLS negotiation progress

* src/gnutls.h (GNUTLS_EMACS_HANDSHAKES_LIMIT): Increase the
number of retries so that we try for about a minute.

* src/process.c (wait_reading_process_output): Ensure progress
for DNS resolution and TLS negotiation.

emacs-module.h slight simplification

* src/emacs-module.c (emacs_init_function, emacs_subr): Move here ...
* src/emacs-module.h: ... from here, as they don’t need to be public.
(enum emacs_arity): Remove useless enum tag.  The enum value is
used in ptrdiff_t contexts.

* src/emacs-module.c (CHECK_USER_PTR): Fix typo in previous change.

* src/emacs-module.c (CHECK_USER_PTR): Fix typo in previous change.

emacs-module.c simplification and tuneup

* src/emacs-module.c (CHECK_USER_PTR): New function.
(module_get_user_ptr, module_set_user_ptr)
(module_get_user_finalizer, module_set_user_finalizer): Use it.
(module_make_global_ref, module_copy_string_contents)
(module_make_string, module_vec_set, module_vec_get): Omit
unnecessary runtime tests.  For example, vector sizes are always
fixnums, so we don’t need to test that they are in fixnum range.

Use standard checks whenever possible.

This is possible in all functions where we catch signals anyway.

* emacs-module.c (module_make_global_ref, module_funcall)
(module_copy_string_contents, module_make_string): Use xsignal0
and CHECK macros for argument checks.

Deterministic build improvements

* configure.ac (BUILD_DETAILS): Rename from DETERMINISTIC_DUMP,
and negate its sense.  Use it via AC_SUBST, not AC_DEFINE,
and have its value be either empty or --no-build-details.
All uses changed.  Change option to --disable-build-details.
* doc/lispref/cmdargs.texi (Initial Options):
Document --no-build-details.
* doc/lispref/internals.texi (Building Emacs):
* etc/NEWS:
Document --disable-build-details.
* doc/lispref/intro.texi (Version Info): Say that
emacs-build-time can be nil.
* lisp/erc/erc-compat.el (erc-emacs-build-time):
Now nil if details are omitted.
* lisp/erc/erc.el (erc-cmd-SV):
* lisp/version.el (emacs-build-time):
Now nil if no build details.
(emacs-version):
Output build time only if build details.
* src/Makefile.in (BUILD_DETAILS): New macro.
(temacs, bootstrap-emacs): Use it.
* src/emacs.c (build_details): New var.
(standard_args, main): Support --no-build-details.
(Vdeterministic_dump): Remove; all uses replaced
by !build_details.
(syms_of_emacs): Set Vbuild_details to a boolean, not
to a Lisp_Object.
* src/lisp.h (build_details): New decl.
* src/sysdep.c (init_system_name): When !build_details,
set system-name to nil, not to "elided".

Remove build system name from deterministic dumps

* configure.ac (DETERMINISTIC_DUMP): New configuration option.
* lisp/version.el (emacs-build-time): Add a comment to make the
build time deterministic if requested.
(emacs-build-system): Make variable deterministic if requested.
* src/emacs.c (main): Initialize `deterministic-dump' from the
configuration option.
(syms_of_emacs): New constant `deterministic-dump'.
* src/sysdep.c (init_system_name): Use a constant
if a deterministic dump is requested.

Remove buggy parse-time test

* test/lisp/calendar/parse-time-tests.el (parse-time-tests):
Remove the iso parse test, since it doesn't work.

* src/syntax.c (syntax_multibyte): Omit unnecessary parens.

; Auto-commit of loaddefs files.

* lisp/emacs-lisp/autoload.el (autoload-timestamps):
Experiment with setting to nil.

; * etc/NEWS: Mention this.

Make the revert command in timer-list-mode work

* lisp/emacs-lisp/timer-list.el (timer-list): Make the revert
command work.

* lisp/emacs-lisp/autoload.el (autoload-find-destination):
Fix 6-week old merge error.

* lisp/emacs-lisp/autoload.el (update-file-autoloads):
Ensure timestamps.

* lisp/emacs-lisp/package.el (package-generate-autoloads):
Disable timestamps.

Allow binding `url-mime-accept-string'

* lisp/url/url-http.el (url-http): Allow binding
`url-mime-accept-string' (bug#22855).

Add a NEWS entry for the colorful VC indicator

Use colors in the VC mode lines

* lisp/vc/vc-hooks.el: Make the mode line faces default to
using colors to more clearly tell the user what the status is.

Support <bdo> and <bdi>

* lisp/net/shr.el (shr-tag-bdo): New function.
(shr-tag-bdi): Ditto.

Respect <html dir=auto>

* lisp/net/shr.el (shr-tag-html): Respect the "auto"
directional HTML setting.

Default bidi paragraph direction to nil

* lisp/net/eww.el (eww-display-html): Default bidi rendering
to nil, so that possibly more Arabic web pages render
correctly (bug#22786).
(eww-setup-buffer): Ditto.

Preserve current column, going up/down in thumbnail view.

Handle "noise" macros and compiler directives.

* lisp/progmodes/cc-langs.el (c-symbol-char-key): New language variable.

* lisp/progmodes/cc-vars.el (c-noise-macro-names)
(c-noise-macro-with-parens-names): New customizable variables.
(c-noise-macro-name-re, c-noise-macro-with-parens-name-re): New variables.
(c-make-noise-macro-regexps): New function.

* lisp/progmodes/cc-engine.el (c-forward-sws, c-backward-sws): Adapt to treat
members of c-noise-macro-names as whitespace.
(c-forward-noise-clause): New function.
(c-forward-keyword-prefixed-id, c-forward-type, c-forward-declarator)
(c-forward-decl-or-cast-1, c-backward-over-enum-header)
(c-guess-basic-syntax CASE 5A.3, CASE 5A.5, CASE 9A):
Handle "noise clauses" in parallel with, e.g., "hangon key clauses".

* lisp/progmodes/cc-fonts.el (c-complex-decl-matchers): Handle "noise clauses"
in parallel with "prefix-spec keywords".

* lisp/progmodes/cc-mode.el (c-mode, c++-mode, objc-mode): call
c-make-noise-macro-regexps to initialize the internal variables.

* doc/misc/cc-mode.texi ("Noise Macros"): New section documenting the new
facilities.

Stop calling res_init

Emacs shouldn’t need to call res_init any more, now that nscd or
equivalent is everywhere.  On modern systems, calling res_init
simply slows Emacs down.  On ancient systems lacking nscd Emacs
will still work well enough with this change; it’s just that it
won’t respond to changes in /etc/resolv.conf.
* configure.ac (HAVE_RES_INIT): Remove.  Worry about -lresolv only
when configured --with-hesiod.  Hesiod is still used; see, e.g.:
https://soylentnews.org/meta/article.pl?sid=15/07/13/0255214
* src/Makefile.in (LIBRESOLV): Remove.  All uses removed.

* lisp/progmodes/grep.el (grep-save-buffers): Fix type, tweak doc.

* lisp/progmodes/grep.el (grep-save-buffers): Don't autoload defcustoms.

; (Unless there is a good, specific reason.)

* src/process.c (Fdelete_process): Simplify cast.

Use the correct background color when filling nested <divs>

* lisp/net/shr.el (shr-face-background): Return the first
background, because that's the one that's visible (bug#22680).

Make <div> in <li> not insert extra newlines

* lisp/net/shr.el (shr-tag-div): Make <div> in <li> not insert
extra newlines (bug#19587).

Allow reading Gnus reports from an offline cache

* lisp/gnus/gnus-group.el (gnus-read-ephemeral-bug-group):
Allow reading bug reports from an offline cache directory.

Meta + shift + Fn combos are recognized by xterm>=216.

* lisp/term/xterm.el (xterm-alternatives-map): Meta + shift +
Fn combos are recognized by xterm>=216 (bug#22837).

Declare $ as an expression prefix in SCSS

* lisp/textmodes/css-mode.el (scss-mode-syntax-table): Declare $ as an
expression prefix (bug#22841).

Revert "Declare $ as an expression prefix in SCSS"

This reverts commit ffb33264f21e8af60b8055c47baffcf0f6c300df.

Revering to fix author name.

Declare $ as an expression prefix in SCSS

* lisp/textmodes/css-mode.el (scss-mode-syntax-table): Declare $ as an
expression prefix (bug#22841).

Save buffers before running grep commands

* lisp/progmodes/grep.el (grep-ask-about-save): New variable (bug#96).
(grep, lgrep, rgrep): Use it (bug#96).

* doc/emacs/building.texi (Grep Searching): Document
`grep-save-buffers'.

* lisp/progmodes/grep.el (grep-save-buffers): Rename from
`grep-ask-about-save'.
(grep--save-buffers): New function.
(grep, lgrep, rgrep): Use it.

Allow sorting flyspell corrections

* lisp/textmodes/flyspell.el (flyspell-sort): New function (bug#2405).
(flyspell-sort-corrections-functionx): New variable.
(flyspell-sort-corrections-alphabetically): New function.
(flyspell-notify-misspell): Use them.
(flyspell-auto-correct-word): Ditto.
(flyspell-emacs-popup): Ditto.
(flyspell-xemacs-popup): Ditto.  Suggested by Sebastien Delafond.

Wait for async DNS to complete before freeing resources

* src/process.c (Fdelete_process): Wait for async DNS to
complete before freeing the data structures it needs.

* lisp/faces.el: Remove duplicated functions from previous patch.

Fix TLS connections on MS-Windows

* src/w32.c (sys_write): If 'send' returns with WSAENOTCONN, and
this is a non-blocking socket whose connection is in progress, set
errno to EWOULDBLOCK, as expected by GnuTLS and other callers.
Avoid overwriting the errno value from 'send' by 'ioctlsocket'.
Suggested by Alain Schneble <a.s@realize.ch>.  (Bug#22789)

Add a NEWS entry for the read-color change

Show the face colours when completing in `read-color'

* lisp/faces.el (defined-colors-with-face-attributes): New function.
(readable-foreground-color, defined-colors-with-face-attributes)
(readable-foreground-color): Ditto.
(read-color): Use them (bug#5305).

Clean up the code in parse-time-string-chars

* lisp/calendar/parse-time.el (parse-time-string-chars):
Clean up the code a bit.

Add some tests for parse-time.el

* test/lisp/calendar/parse-time-tests.el: New file.

Strip out some leading whitespace when looking at logs

* lisp/vc/vc-git.el (vc-git-expanded-log-entry): When looking
at expanded git logs with `vc-print-root-log' (C-x v L, then
<enter> by default), Emacs was stripping out all leading
whitespace from git logs.  I now strip exactly 2 leading
spaces, which retains the indentation in the logs (bug#18110).

Use a separate history variable for align-regexp

* lisp/align.el (align-regexp-history): New variable (bug#16891).
(align-regexp): Use it.

Create subdirectories automatically in wdired

* lisp/wdired.el (wdired-create-parent-directories): New
variable (bug#6817).
(wdired-create-parentdirs): New function.
(wdired-do-renames): Use it.
* doc/emacs/dired.texi (Wdired): Mention
`wdired-create-parent-directories'

Return the correct error values from gnutls.c

* src/gnutls.c (emacs_gnutls_read): Set errno to the value
expected by process.c.
(gnutls_try_handshake): Set gnutls_p to true earlier to avoid
possible race condition with the process.c socket polling
functions.

Don't make assumptions about mkdir.  (Bug#22822)

* test/Makefile.in (MKDIR_P): New, set by configure.
(%.log): Use MKDIR_P.

Support switching to hexl-mode from image-mode

* lisp/image-mode.el (image-toggle-hex-display)
(image-mode-to-text): New functions.
(image-mode-map, image-minor-mode-map): Bind "C-c C-x" to
image-toggle-hex-display.
(image-mode-map): New menu item "Show as Hex".
(image-mode): Update doc string and echo-area message.
(image-toggle-display): Support toggle to hex.  (Bug#22453)

* doc/emacs/files.texi (File Conveniences): Document
'image-toggle-hex-display'.

Remove unused code in coding.c

* src/coding.c (decode_eol): Remove unused code that handled the
case of coding->dst_object being nil.  Replace it with an
assertion.

* src/syntax.c: Fix result of syntax_multibyte()

The ?: operator has a lower precedence than ||.
Bug introduced by commit 45b683a.

--enable-gcc-warnings now uses -Wjump-misses-init

When configuring with --enable-gcc-warnings, also enable
-Wjump-misses-init, as it’s confusing to use a goto to skip over
an initialization.  Fix the few places in the code that run afoul
of this warning.
* configure.ac (WERROR_CFLAGS): Add -Wjump-misses-init.
* src/doc.c (Fsubstitute_command_keys):
* src/image.c (svg_load_image):
* src/regex.c (re_match_2_internal):
* src/xdisp.c (redisplay_internal, redisplay_window):
Don’t jump over initialization.

; Improve documentation in test Makefile

* test/Makefile.in: Improve documentation.

Improve the register-hotkey functionality on MS-Windows

* src/w32fns.c (_WIN32_WINNT): Define to 0x0600, needed for
keyboard hook functionality.
Include w32inevt.h, basetyps.h and unknwn.h.
(VK_ANY, WM_WTSSESSION_CHANGE, WTS_SESSION_LOCK): New macros.
(kbdhook): A new struct definition.
(funhook, setup_w32_kbdhook, remove_w32_kbdhook, hook_w32_key)
(check_w32_winkey_state, reset_w32_kbdhook_state): New functions.
(modifier_set): Call check_w32_winkey_state if a Win key was
pressed and the keyboard hook is active.
(w32_wnd_proc): Don't handle Win key combinations if the keyboard
hook is active.  Only register/unregister the hotkeys if the
keyboard hook is not active.  When WM_CREATE is received, call
setup_w32_kbdhook.  When WM_DESTROY is received, call
reset_w32_kbdhook_state.
(lookup_vk_code): When the keyboard hook is active, map
alphanumeric characters to themselves.
(w32_parse_and_hook_hot_key): Renamed from w32_parse_hot_key.  Map
modified keys to VK_ANY if the keyboard hook is active.  Register
Alt-x and Win-x combinations.
(Fw32_shell_execute): Update doc string to reflect new
functionality.  Bypass the code that posts the
WM_EMACS_REGISTER_HOT_KEY message if the keyboard hook is active.
(Fw32_unregister_hot_key): Bypass the code that posts the
WM_EMACS_UNREGISTER_HOT_KEY message if the keyboard hook is active.
(syms_of_w32fns) <w32-pass-lwindow-to-system>
<w32-pass-rwindow-to-system, w32-phantom-key-code>
<w32-lwindow-modifier, w32-rwindow-modifier>: Update doc strings
to reflect the new functionality.
* src/w32console.c (initialize_w32_display): Install the low-level
keyboard hook.
* src/w32inevt.c (key_event): Handle Win-x combinations only if
the keyboard hook is not active.  If the hook is active, use
check_w32_winkey_state instead.
* src/w32term.h (setup_w32_kbdhook, remove_w32_kbdhook)
(check_w32_winkey_state): Add prototypes.
(w32_kbdhook_active): New macro.

* doc/emacs/msdos.texi (Windows Keyboard): Update to reflect the
new functionality.

Add forward-sexp (and related) tests

* test/lisp/emacs-lisp/lisp-tests.el: New file for testing
forward-sexp and related functions (bug#22800).

Don't use (localised) week days in dates

* lisp/gnus/gnus-icalendar.el
(gnus-icalendar-event:org-timestamp): Don't use (localised)
week days in the dates, because that messes up things later.

Update pcomplete/find

* lisp/pcmpl-gnu.el (pcomplete/find): Update to newest version
(bug#10487).

Make parse-time-string-chars more efficient

* lisp/calendar/parse-time.el (parse-time-string-chars): The
string has already been downcase, so don't care about case
(bug#18522).

Remove killed buffers from the list of Gnus buffers

* lisp/gnus/gnus.el (gnus-prune-buffers): Remove killed
buffers from the list of Gnus buffers (bug#18522).

Integer overflow cleanups for ports and socklen

* src/process.c (struct sockaddr_and_len, conv_sockaddr_to_lisp)
(get_lisp_to_sockaddr_size, Fset_process_datagram_address)
(connect_network_socket):
Use ptrdiff_t, not int, for signed object sizes.
This addresses only a theoretical problem, as in practice these
object sizes are less than 2**31, but we might as well use the
same style here as elsewhere in Emacs.
(string_integer_p): Remove; all uses removed.
(Fmake_network_process): Check that port number is in range.
When converting an integer-string service, rely on strtol
rather than rechecking the string by hand.
* src/process.h, src/w32.c (conv_sockaddr_to_lisp):
Adjust prototypes to match.

Add pcomplete support for find

* lisp/pcmpl-gnu.el (pcomplete/find): Add pcomplete support
for find (bug#10487).

Replace XXX acronyms with draft standard ones

* lisp/international/characters.el (c1-acronyms): Replace XXX
entries with the acronyms PAD, HOP, SGCI from draft DIS 10646
(bug#13745).

Make checkdoc warn about variables described as "True"

* checkdoc.el (checkdoc-this-string-valid-engine): Docstrings for
variables "True...", and functions "Return true...", should usually be
"non-nil" (bug#15506).

Add NEWS entry for "number string" change

Allow using "number strings" as services on non-GNU systems

* src/process.c (string_integer_p): New function.
(Fmake_network_process): Use it to allow connecting to
services specified as "993" even when getaddrbyname isn't
available.

Minor fixes in calculator.el

* lisp/calculator.el (calculator-mode-map): Bind `E' for
`calculator-exp'.
(calculator-last-input): Fix a bug with pressing F1.
(Bug#20764)

Copyright-paperwork-exempt: yes

Update HISTORY section in readme for the NextStep interface.

* nextstep/README: Update HISTORY after suggestion
from former maintainer Adrian Robert.

; * doc/lispref/modes.texi (Font Lock Basics): Minor rewording.

Improve documentation of 'save-place-mode'

* lisp/saveplace.el (toggle-save-place): Update the doc string wrt
turning on 'save-place-mode' globally.

* etc/NEWS: Mention the need to call 'save-place-mode' for turning
on the mode in all buffers.

Fix problem in tramp.texi

* doc/misc/tramp.texi (File name completion): Do not use
@trampfn{} for IPv6 addresses.  Somehow, it results in errors
during PDF creation.

Allocate glyph matrices for the initial frame

* src/frame.c (make_initial_frame): Allocate glyph matrices (Bug#22787).

* src/dispnew.c (clear_glyph_matrix_rows): matrix->nrows can be 0.

Copyright-paperwork-exempt: yes

Make image commands available in image-mode

* lisp/image-mode.el (image-mode-map): Inherit from
`image-map' so that the usual image commands work.

Move `o' to `O' in shr-map

* lisp/net/shr.el (shr-map): `shr-save-contents' has moved
from `o' to `O' to avoid collisions with `image-map'.

Make setting the coding system non-blocking

* src/process.c (Fset_process_filter_multibyte): Defer
completing coding system setup in asynchronous processes.
(Fset_process_coding_system): Ditto.

Mention `proced-toggle-auto-update' in the proced-mode doc string

* lisp/proced.el (proced-mode): Mention
`proced-toggle-auto-update' (bug#14341).

Add a fortune-message command

* lisp/play/fortune.el (fortune-message): Add a command to
display fortunes in the echo area (bug#14915).

Copyright-paperwork-exempt: yes

Add hideshow bindings analogous to outline mode

* lisp/progmodes/hideshow.el (hs-minor-mode-map): Add bindings
analogous to outline mode bindings (bug#15324).

Copyright-paperwork-exempt: yes

Allow highlighting things like @math{2^{12}}

* lisp/textmodes/texinfo.el (texinfo-font-lock-keywords):
Allow highlighting things like @math{2^{12}} (bug#16390).

Make `C-{up,down,left,right}' work in term mode

* lisp/term.el (term-raw-map): Define C-{up,down,left,right}
to send the same escape sequence that xterm sends.  This makes
backward-word and forward-word work in readline (bug#16746).

Allow ff-find-other-file (etc) to work with indirect clone buffers

* lisp/find-file.el (ff-buffer-file-name): New function to
allow the feature to work with indirect buffers, too
(bug#16904).
(ff-find-the-other-file): Use it.
(ff-other-file-name): Ditto.
(ff-get-file-name): Ditto.

Fix white space in last checkin

Make `insert-pair' always leave the cursor where documented

* lisp/emacs-lisp/lisp.el (insert-pair): The docstring of
insert-pair states that after insertion, the point ends up
after the opening character. This was not true if the pair was
inserted to surround a region (bug#16949).

etc/NEWS: Mention the new second parameter to `package-install'

* etc/NEWS: Mention the new second parameter to
`package-install' (bug#22784).

Fix error in add-change-log-entry when the entry has no newline

* lisp/vc/add-log.el (add-change-log-entry): The entry in the
ChangeLog may end without a new line starting with blanks.

Avoid using mm-util functoins in gravatar.el

* lisp/image/gravatar.el (gravatar-retrieve): Avoid using
mm-util functions.
(gravatar-retrieve-synchronously): Ditto.

Move low-level library files from the lisp/gnus directory

The files moved from lisp/gnus are:

auth-source.el -> /
compface.el -> /image
ecomplete.el -> /
flow-fill.el -> /mail
gravatar.el -> /image
gssapi.el -> /net
html2text.el -> /net
ietf-drums.el -> /mail
mail-parse.el -> /mail
mail-prsvr.el -> /mail
mailcap.el -> /net
plstore.el -> /
pop3.el -> /net
qp.el -> /mail
registry.el -> /
rfc1843.el -> /international
rfc2045.el -> /mail
rfc2047.el -> /mail
rfc2231.el -> /mail
rtree.el -> /
sieve-manage.el -> /net
sieve-mode.el -> /net
sieve.el -> /net
starttls.el -> /net
utf7.el -> /international
yenc.el -> /mail

Allow undoing changes while doing query-replace

* doc/lispref/searching.texi (Search and Replace): Mention
undo (bug#21684).

* lisp/replace.el (query-replace-help): Document undo.
(perform-replace): Implement undo while replacing text.

Highlight assignments in Makefiles more correctly

* lisp/progmodes/make-mode.el (makefile-macroassign-regex):
Highlight assignments preceded by a TAB character correctly
(bug#20787).

Copyright-paperwork-exempt: yes

Backport:

(cherry picked from commit bbd86c5642bd62c43d72391669f28eaa14459fd5)

Highlight assignments in Makefiles more correctly

* lisp/progmodes/make-mode.el (makefile-macroassign-regex):
Highlight assignments preceded by a TAB character correctly
(bug#20787).

Copyright-paperwork-exempt: yes

Use memmove instead of memcpy on overlapping regions

* src/editfns.c (Ftranspose_regions): Regions may overlap, so
use memmove instead of memcpy (bug#19213).

Copyright-paperwork-exempt: yes

Port to --enable-gcc-warnings sans getaddrinfo_a

* src/process.c (Fmake_network_process): Add ATTRIBUTE_UNUSED
to a local unused when getaddrinfo_a is missing.
Resize portbuf to size needed.  Do cheap test first.
Move local to block where it’s needed.